Fix issues with learner record api by annagav · Pull Request #3600 · mitodl/mitxonline

annagav · 2026-05-21T18:06:32Z

What are the relevant tickets?

Fix https://github.com/mitodl/hq/issues/10918

Description (What does it do?)

Fix issues with learner record api

How can this be tested?

Create partner school
Go to http://mitxonline.odl.local:8013/api/records/program/2/ and verify the under "partner_schools" email field is not there.
Make sure you can't view your program record if you don't have an enrollment

github-actions · 2026-05-21T18:06:52Z

OpenAPI Changes

Show/hide ## Changes for v0.yaml:

## Changes for v0.yaml:
22 changes: 12 error, 10 warning, 0 info
error	[response-required-property-removed] at head/openapi/specs/v0.yaml	
	in API GET /api/records/program/{id}/
		removed the required property `partner_schools/items/created_on` from the response with the `200` status

error	[response-required-property-removed] at head/openapi/specs/v0.yaml	
	in API GET /api/records/program/{id}/
		removed the required property `partner_schools/items/email` from the response with the `200` status

error	[response-required-property-removed] at head/openapi/specs/v0.yaml	
	in API GET /api/records/program/{id}/
		removed the required property `partner_schools/items/updated_on` from the response with the `200` status

error	[response-required-property-removed] at head/openapi/specs/v0.yaml	
	in API POST /api/records/program/{id}/revoke/
		removed the required property `partner_schools/items/created_on` from the response with the `200` status

error	[response-required-property-removed] at head/openapi/specs/v0.yaml	
	in API POST /api/records/program/{id}/revoke/
		removed the required property `partner_schools/items/email` from the response with the `200` status

error	[response-required-property-removed] at head/openapi/specs/v0.yaml	
	in API POST /api/records/program/{id}/revoke/
		removed the required property `partner_schools/items/updated_on` from the response with the `200` status

error	[response-required-property-removed] at head/openapi/specs/v0.yaml	
	in API POST /api/records/program/{id}/share/
		removed the required property `partner_schools/items/created_on` from the response with the `200` status

error	[response-required-property-removed] at head/openapi/specs/v0.yaml	
	in API POST /api/records/program/{id}/share/
		removed the required property `partner_schools/items/email` from the response with the `200` status

error	[response-required-property-removed] at head/openapi/specs/v0.yaml	
	in API POST /api/records/program/{id}/share/
		removed the required property `partner_schools/items/updated_on` from the response with the `200` status

error	[response-required-property-removed] at head/openapi/specs/v0.yaml	
	in API GET /api/records/shared/{uuid}/
		removed the required property `partner_schools/items/created_on` from the response with the `200` status

error	[response-required-property-removed] at head/openapi/specs/v0.yaml	
	in API GET /api/records/shared/{uuid}/
		removed the required property `partner_schools/items/email` from the response with the `200` status

error	[response-required-property-removed] at head/openapi/specs/v0.yaml	
	in API GET /api/records/shared/{uuid}/
		removed the required property `partner_schools/items/updated_on` from the response with the `200` status

warning	[response-optional-property-removed] at head/openapi/specs/v0.yaml	
	in API GET /api/records/program/{id}/
		removed the optional property `partner_schools/items/is_active` from the response with the `200` status

warning	[response-optional-property-removed] at head/openapi/specs/v0.yaml	
	in API POST /api/records/program/{id}/revoke/
		removed the optional property `partner_schools/items/is_active` from the response with the `200` status

warning	[request-property-removed] at head/openapi/specs/v0.yaml	
	in API POST /api/records/program/{id}/share/
		removed the request property `email` (media type: application/json)

warning	[request-property-removed] at head/openapi/specs/v0.yaml	
	in API POST /api/records/program/{id}/share/
		removed the request property `email` (media type: multipart/form-data)

warning	[request-property-removed] at head/openapi/specs/v0.yaml	
	in API POST /api/records/program/{id}/share/
		removed the request property `email` (media type: application/x-www-form-urlencoded)

warning	[request-property-removed] at head/openapi/specs/v0.yaml	
	in API POST /api/records/program/{id}/share/
		removed the request property `is_active` (media type: multipart/form-data)

warning	[request-property-removed] at head/openapi/specs/v0.yaml	
	in API POST /api/records/program/{id}/share/
		removed the request property `is_active` (media type: application/x-www-form-urlencoded)

warning	[request-property-removed] at head/openapi/specs/v0.yaml	
	in API POST /api/records/program/{id}/share/
		removed the request property `is_active` (media type: application/json)

warning	[response-optional-property-removed] at head/openapi/specs/v0.yaml	
	in API POST /api/records/program/{id}/share/
		removed the optional property `partner_schools/items/is_active` from the response with the `200` status

warning	[response-optional-property-removed] at head/openapi/specs/v0.yaml	
	in API GET /api/records/shared/{uuid}/
		removed the optional property `partner_schools/items/is_active` from the response with the `200` status



## Changes for v1.yaml:
22 changes: 12 error, 10 warning, 0 info
error	[response-required-property-removed] at head/openapi/specs/v1.yaml	
	in API GET /api/records/program/{id}/
		removed the required property `partner_schools/items/created_on` from the response with the `200` status

error	[response-required-property-removed] at head/openapi/specs/v1.yaml	
	in API GET /api/records/program/{id}/
		removed the required property `partner_schools/items/email` from the response with the `200` status

error	[response-required-property-removed] at head/openapi/specs/v1.yaml	
	in API GET /api/records/program/{id}/
		removed the required property `partner_schools/items/updated_on` from the response with the `200` status

error	[response-required-property-removed] at head/openapi/specs/v1.yaml	
	in API POST /api/records/program/{id}/revoke/
		removed the required property `partner_schools/items/created_on` from the response with the `200` status

error	[response-required-property-removed] at head/openapi/specs/v1.yaml	
	in API POST /api/records/program/{id}/revoke/
		removed the required property `partner_schools/items/email` from the response with the `200` status

error	[response-required-property-removed] at head/openapi/specs/v1.yaml	
	in API POST /api/records/program/{id}/revoke/
		removed the required property `partner_schools/items/updated_on` from the response with the `200` status

error	[response-required-property-removed] at head/openapi/specs/v1.yaml	
	in API POST /api/records/program/{id}/share/
		removed the required property `partner_schools/items/created_on` from the response with the `200` status

error	[response-required-property-removed] at head/openapi/specs/v1.yaml	
	in API POST /api/records/program/{id}/share/
		removed the required property `partner_schools/items/email` from the response with the `200` status

error	[response-required-property-removed] at head/openapi/specs/v1.yaml	
	in API POST /api/records/program/{id}/share/
		removed the required property `partner_schools/items/updated_on` from the response with the `200` status

error	[response-required-property-removed] at head/openapi/specs/v1.yaml	
	in API GET /api/records/shared/{uuid}/
		removed the required property `partner_schools/items/created_on` from the response with the `200` status

error	[response-required-property-removed] at head/openapi/specs/v1.yaml	
	in API GET /api/records/shared/{uuid}/
		removed the required property `partner_schools/items/email` from the response with the `200` status

error	[response-required-property-removed] at head/openapi/specs/v1.yaml	
	in API GET /api/records/shared/{uuid}/
		removed the required property `partner_schools/items/updated_on` from the response with the `200` status

warning	[response-optional-property-removed] at head/openapi/specs/v1.yaml	
	in API GET /api/records/program/{id}/
		removed the optional property `partner_schools/items/is_active` from the response with the `200` status

warning	[response-optional-property-removed] at head/openapi/specs/v1.yaml	
	in API POST /api/records/program/{id}/revoke/
		removed the optional property `partner_schools/items/is_active` from the response with the `200` status

warning	[request-property-removed] at head/openapi/specs/v1.yaml	
	in API POST /api/records/program/{id}/share/
		removed the request property `email` (media type: application/json)

warning	[request-property-removed] at head/openapi/specs/v1.yaml	
	in API POST /api/records/program/{id}/share/
		removed the request property `email` (media type: multipart/form-data)

warning	[request-property-removed] at head/openapi/specs/v1.yaml	
	in API POST /api/records/program/{id}/share/
		removed the request property `email` (media type: application/x-www-form-urlencoded)

warning	[request-property-removed] at head/openapi/specs/v1.yaml	
	in API POST /api/records/program/{id}/share/
		removed the request property `is_active` (media type: multipart/form-data)

warning	[request-property-removed] at head/openapi/specs/v1.yaml	
	in API POST /api/records/program/{id}/share/
		removed the request property `is_active` (media type: application/x-www-form-urlencoded)

warning	[request-property-removed] at head/openapi/specs/v1.yaml	
	in API POST /api/records/program/{id}/share/
		removed the request property `is_active` (media type: application/json)

warning	[response-optional-property-removed] at head/openapi/specs/v1.yaml	
	in API POST /api/records/program/{id}/share/
		removed the optional property `partner_schools/items/is_active` from the response with the `200` status

warning	[response-optional-property-removed] at head/openapi/specs/v1.yaml	
	in API GET /api/records/shared/{uuid}/
		removed the optional property `partner_schools/items/is_active` from the response with the `200` status



## Changes for v2.yaml:
22 changes: 12 error, 10 warning, 0 info
error	[response-required-property-removed] at head/openapi/specs/v2.yaml	
	in API GET /api/records/program/{id}/
		removed the required property `partner_schools/items/created_on` from the response with the `200` status

error	[response-required-property-removed] at head/openapi/specs/v2.yaml	
	in API GET /api/records/program/{id}/
		removed the required property `partner_schools/items/email` from the response with the `200` status

error	[response-required-property-removed] at head/openapi/specs/v2.yaml	
	in API GET /api/records/program/{id}/
		removed the required property `partner_schools/items/updated_on` from the response with the `200` status

error	[response-required-property-removed] at head/openapi/specs/v2.yaml	
	in API POST /api/records/program/{id}/revoke/
		removed the required property `partner_schools/items/created_on` from the response with the `200` status

error	[response-required-property-removed] at head/openapi/specs/v2.yaml	
	in API POST /api/records/program/{id}/revoke/
		removed the required property `partner_schools/items/email` from the response with the `200` status

error	[response-required-property-removed] at head/openapi/specs/v2.yaml	
	in API POST /api/records/program/{id}/revoke/
		removed the required property `partner_schools/items/updated_on` from the response with the `200` status

error	[response-required-property-removed] at head/openapi/specs/v2.yaml	
	in API POST /api/records/program/{id}/share/
		removed the required property `partner_schools/items/created_on` from the response with the `200` status

error	[response-required-property-removed] at head/openapi/specs/v2.yaml	
	in API POST /api/records/program/{id}/share/
		removed the required property `partner_schools/items/email` from the response with the `200` status

error	[response-required-property-removed] at head/openapi/specs/v2.yaml	
	in API POST /api/records/program/{id}/share/
		removed the required property `partner_schools/items/updated_on` from the response with the `200` status

error	[response-required-property-removed] at head/openapi/specs/v2.yaml	
	in API GET /api/records/shared/{uuid}/
		removed the required property `partner_schools/items/created_on` from the response with the `200` status

error	[response-required-property-removed] at head/openapi/specs/v2.yaml	
	in API GET /api/records/shared/{uuid}/
		removed the required property `partner_schools/items/email` from the response with the `200` status

error	[response-required-property-removed] at head/openapi/specs/v2.yaml	
	in API GET /api/records/shared/{uuid}/
		removed the required property `partner_schools/items/updated_on` from the response with the `200` status

warning	[response-optional-property-removed] at head/openapi/specs/v2.yaml	
	in API GET /api/records/program/{id}/
		removed the optional property `partner_schools/items/is_active` from the response with the `200` status

warning	[response-optional-property-removed] at head/openapi/specs/v2.yaml	
	in API POST /api/records/program/{id}/revoke/
		removed the optional property `partner_schools/items/is_active` from the response with the `200` status

warning	[request-property-removed] at head/openapi/specs/v2.yaml	
	in API POST /api/records/program/{id}/share/
		removed the request property `email` (media type: application/json)

warning	[request-property-removed] at head/openapi/specs/v2.yaml	
	in API POST /api/records/program/{id}/share/
		removed the request property `email` (media type: multipart/form-data)

warning	[request-property-removed] at head/openapi/specs/v2.yaml	
	in API POST /api/records/program/{id}/share/
		removed the request property `email` (media type: application/x-www-form-urlencoded)

warning	[request-property-removed] at head/openapi/specs/v2.yaml	
	in API POST /api/records/program/{id}/share/
		removed the request property `is_active` (media type: multipart/form-data)

warning	[request-property-removed] at head/openapi/specs/v2.yaml	
	in API POST /api/records/program/{id}/share/
		removed the request property `is_active` (media type: application/x-www-form-urlencoded)

warning	[request-property-removed] at head/openapi/specs/v2.yaml	
	in API POST /api/records/program/{id}/share/
		removed the request property `is_active` (media type: application/json)

warning	[response-optional-property-removed] at head/openapi/specs/v2.yaml	
	in API POST /api/records/program/{id}/share/
		removed the optional property `partner_schools/items/is_active` from the response with the `200` status

warning	[response-optional-property-removed] at head/openapi/specs/v2.yaml	
	in API GET /api/records/shared/{uuid}/
		removed the optional property `partner_schools/items/is_active` from the response with the `200` status

Unexpected changes? Ensure your branch is up-to-date with main (consider rebasing).

for more information, see https://pre-commit.ci

Fix issues with learner record api

a195431

annagav added 2 commits May 22, 2026 13:29

spec

8adb407

format

8f2d52a

annagav force-pushed the ag/learner_record_api branch from 846563c to 8f2d52a Compare May 22, 2026 17:36

[pre-commit.ci] auto fixes from pre-commit.com hooks

ae2e738

for more information, see https://pre-commit.ci

annagav added the Needs Review label May 22, 2026

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Fix issues with learner record api#3600

Fix issues with learner record api#3600
annagav wants to merge 4 commits into
mainfrom
ag/learner_record_api

annagav commented May 21, 2026 •

edited

Loading

Uh oh!

github-actions Bot commented May 21, 2026 •

edited

Loading

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Conversation

annagav commented May 21, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

What are the relevant tickets?

Description (What does it do?)

How can this be tested?

Uh oh!

github-actions Bot commented May 21, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

OpenAPI Changes

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

annagav commented May 21, 2026 •

edited

Loading

github-actions Bot commented May 21, 2026 •

edited

Loading